Skip to content

[pull] dev from main - #372

Open
pull[bot] wants to merge 16 commits into
devfrom
main
Open

[pull] dev from main#372
pull[bot] wants to merge 16 commits into
devfrom
main

Conversation

@pull

@pull pull Bot commented Aug 28, 2026

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )


Note

High Risk
Touches production deploy config, payments reconciliation, event check-in concurrency, admin authorization, and a new GCS-backed PII surface; incorrect cache or reconcile logic could affect live members.

Overview
This PR ships member resume storage in Cloud Storage (metadata in Postgres, upload/serve/ZIP via Next route handlers, staff /admin/resumes, RESUME_BUCKET in App Hosting) and documents the club production site (docs/club-project.md, resume-book docs, glossary/README updates). Club project applications now require a pitch and can attach an optional PDF resume; leaders fetch resumes via a dedicated procedure.

Tail latency and load get a large pass: CacheService gains getOrSet with in-flight deduplication and correct caching of null answers; hot routers (events, hackathons, member, admin analytics, portal context) adopt it. Event check-in only takes FOR UPDATE when a capacity cap exists; Stripe reconcileMyPayments batches DB reads; mass email paths use bounded SMTP concurrency (forEachWithConcurrency). Metrics add p99 histogram buckets, Prometheus recording rules/alerts, Grafana p99 panels, and METRICS_TOKEN for /api/metrics.

Security and repo hygiene: expanded .gitignore, new .githooks (block secrets/agent dirs; block Claude attribution in commits), and contributing guidance. CI stops pinning pnpm so it follows packageManager / pnpm-workspace.yaml. audit tRPC router is removed (tests dropped); admin.isAdmin excludes volunteers; new admin.growth insights query.

pnpm 12 is reflected in getting-started; TODO.md tracks deploy and follow-ups.

Reviewed by Cursor Bugbot for commit 126b7b7. Bugbot is set up for automated code reviews on this repo. Configure here.

@pull pull Bot locked and limited conversation to collaborators Aug 28, 2026
@pull pull Bot added the ⤵️ pull label Aug 28, 2026
@pull pull Bot assigned aamoghS Aug 28, 2026
@pull
pull Bot requested a review from aamoghS August 28, 2026 23:20
@github-actions github-actions Bot added the dependencies Pull requests that update a dependency file label Sep 6, 2026
aamoghS and others added 10 commits September 6, 2026 15:26
Stop decodeURIComponent from crashing Settings on a truncated filename, wait for the matching ZIP entry instead of the previous one, and keep volunteer accounts from counting as staff. Analytics now uses unexpired memberships and chronological bootcamp terms.
…sues-9d3d

Fix resume book, analytics, and volunteer-admin bugs
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

⤵️ pull dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant